Skip to content

Conversation

@sbraz
Copy link
Contributor

@sbraz sbraz commented Oct 25, 2025

If the user defines a value for tic_command_whitelist and then clears
it, it remains in the configuration as an empty string.
An empty string should be the same as no value at all. Before this
change, when the string was empty, no attributes were polled.

Let's also suggest a Z2M restart in the description, it's the easiest way to
restart the poll.

@sbraz sbraz changed the title fix: LiXee ZLinky: handle empty TIC commald list fix: LiXee ZLinky: handle empty TIC command list Oct 25, 2025
@sbraz sbraz force-pushed the zlinky_tic_whitelist branch 2 times, most recently from 59bc1a9 to a95244d Compare October 25, 2025 20:37
e
.text("tic_command_whitelist", ea.SET)
.withDescription("List of TIC commands to be exposed (separated by comma). Reconfigure device after change. Default: all"),
.withDescription("List of TIC commands to be exposed (separated by comma). Requires Z2M restart. Default: all"),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Koenkk I found that changing the poll interval triggers an update of the list but I think it's better to just suggest a restart.

If the user defines a value for tic_command_whitelist and then clears
it, it remains in the configuration as an empty string.
An empty string should be the same as no value at all. Before this
change, when the string was empty, no attributes were polled.

Let's also suggest a Z2M restart in the description, it's the easiest way to
restart the poll.
@sbraz sbraz force-pushed the zlinky_tic_whitelist branch from a95244d to a8560e8 Compare October 25, 2025 20:43
@Koenkk Koenkk merged commit 68ee1bb into Koenkk:master Oct 26, 2025
3 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Oct 26, 2025

Thanks!

jacky202509 pushed a commit to jacky202509/zigbee-herdsman-converters that referenced this pull request Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants